Skip to content

Feat/codex auth compact support#11

Open
kiroo92 wants to merge 2 commits intoArron196:mainfrom
kiroo92:feat/codex-auth-compact-support
Open

Feat/codex auth compact support#11
kiroo92 wants to merge 2 commits intoArron196:mainfrom
kiroo92:feat/codex-auth-compact-support

Conversation

@kiroo92
Copy link
Copy Markdown

@kiroo92 kiroo92 commented Mar 24, 2026

Summary

  • add Codex auth-backed /responses/compact request normalization
  • preserve compact-specific auth/request headers for Codex execution
  • add regression coverage for compact auth path behavior

Testing

  • go test ./internal/runtime/executor
  • go test ./sdk/api/handlers/openai

@Arron196
Copy link
Copy Markdown
Owner

只保留 inputinstructionsprevious_response_id 应该不太合适,并且全局不再把 prompt_cache_key 写回 body,这个改动范围太大了,我查了下openai的官方文档prompt_cache_key 本身就是 compact 的正式请求字段,如果没有明确的抓包验证,这里不能轻易移除

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants